-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use XDG_CACHE_HOME
for PYLINTHOME
#4661
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you for implementing this change ! I think there is no breaking changes and it can go in 2.10.0
(ie: we'll merge only when all 2.9.x crashes have been fixed).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thanks a lot ! I'm going to merge as soon as we release the last expected 2.9.x
Thank you again for this, it will be available in |
This seems to also (truly) close #1364 |
FYI, appdirs is effectively unmaintained; platformdirs is the community's forked replacement. Black has already switched, and given this dep was newly added, it would make sense to use the modern, maintained replacement now given its drop-in compatible and fixes numerous long-outstanding bugs with the original, instead of having to switch later if and when things eventually break. |
Thanks @CAM-Gerlach I created #4886 for this. |
Steps
doc/whatsnew/<current release.rst>
.Description
On Linux, analysis date are stored under
~/.pylint.d
at the moment, which is not compliant with XDG Base Directory Specification.As @blueyed commented, I believe pickle files should be located under
$XDG_CACHE_HOME
rather than$XDG_DATA_HOME
.Closes #3878.
Type of Changes
Related Issue